Skip to content

ci: disable semantic-release github comment/label hooks#80

Merged
bartzbeielstein merged 1 commit into
mainfrom
fix/release-disable-issue-comments
May 28, 2026
Merged

ci: disable semantic-release github comment/label hooks#80
bartzbeielstein merged 1 commit into
mainfrom
fix/release-disable-issue-comments

Conversation

@bartzbeielstein
Copy link
Copy Markdown
Contributor

Summary

The release workflow on the last two merges to main reported "all jobs failed" even though v0.12.1 and v0.12.2 were both fully published (tag, GitHub release, wheel + sdist all uploaded). The non-zero exit came from the post-publish success hook of @semantic-release/github:

✘ An error occurred while running semantic-release: Error: Could not resolve to an Issue with the number of 70.
{ type: 'NOT_FOUND', path: [ 'repository', 'issue70' ] ... }

@semantic-release/github walks every #N token in the generated release notes and tries to look each one up via GraphQL repository.issue(number: N). That resolver only finds issues, not pull requests. My PR #79 commit body referenced PRs #70#77 (the closed Dependabot PRs), and the lookup 404'd after publish, breaking the workflow exit code.

This PR sets three options on @semantic-release/github so the same class of failure cannot happen again:

Option Value Effect
successComment false Stops the post-publish "released in vX.Y.Z" comments that triggered the bug.
failComment false Stops the symmetric post-failure issue creation, which would hit the same lookup path.
releasedLabels false Stops the post-publish label adds on resolved references.

Artifact publishing, tagging, the CHANGELOG/version-bump commit, and the conventional-commit-driven release decision are all unaffected — those happen earlier in the pipeline.

Test plan

  • .releaserc.json parses as valid JSON.
  • Next release on main exits zero (no follow-up "all jobs failed" notification).

The commit is typed ci: so semantic-release will not cut a release for this change itself.

🤖 Generated with Claude Code

The @semantic-release/github post-publish "success" hook walks every
#N reference in the release notes and resolves each via
GraphQL repository.issue(number:). That resolver only finds issues,
not pull requests, so any commit body that references a PR by #N
(e.g. "PRs #70-#77") causes a NOT_FOUND error after the release is
already published — surfacing as a spurious workflow failure.

Setting successComment, failComment, and releasedLabels to false
disables the comment/label features (which the project does not rely
on) and removes the failure mode. Artifact publishing, tagging, and
the CHANGELOG/version-bump commit are unaffected.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@bartzbeielstein bartzbeielstein merged commit 09238e6 into main May 28, 2026
3 checks passed
@bartzbeielstein bartzbeielstein deleted the fix/release-disable-issue-comments branch May 28, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant